Skip to content

feat: L2ToL2CDM SentMessage ctx #266

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

hamdiallam
Copy link
Contributor

@hamdiallam hamdiallam commented Apr 16, 2025

Additional context

Additional of internal context that can be propogated across N cross domain messages

Example High level approach to incentivized relays: https://www.notion.so/oplabs/Incentivized-Message-Relays-1b2f153ee162800e9f36ceab34efa7d9?pvs=4

@hamdiallam hamdiallam changed the title fea: L2ToL2CDM RelayedMessageGasReceipt + SentMessage ctx feat: L2ToL2CDM RelayedMessageGasReceipt + SentMessage ctx Apr 16, 2025
@hamdiallam hamdiallam assigned hamdiallam and unassigned op-aaron Apr 16, 2025
@hamdiallam
Copy link
Contributor Author

Notes from Design Review: Review also covered incentivized delivery in #272

General:

  • all in favor of adding the context opaque bytes. We can do this ahead of time since it is an api breaking change with confidence due to it's general usefulness
  • GasTanks are abstracted enough that can support payments for senders, msgHash, and more. (they can also refund the relayers on origin, destination or any other superchain).
  • Not entirely sure about call depth in the propagated ctx, but we need a way to being able to:
    • reference the parent msg-hash on child sentMessages
    • We might be able to infer ordering of relayed messages on the GasTank via a chain of hashes with the parent message
  • all in favor of adding the RelayedMessageGasReceipt event with minor modifications to be future-proof
  • important we need to go through integration and dapp examples to see where this works and breaks.

Next Steps:

  1. Update this document scoped to just propagating opaque context bytes with the SentMessage event. This will be empty to start
  2. Update or create a new document for RelayedMessageGasReceipt which can be tied with feat: incentivized cross domain message delivery #272. When some prototypes of the gas tank is created, we can then make use of the context bytes with a first version (tx.origin, rootMsgHash) and emit the the gas receipt event.

@hamdiallam hamdiallam changed the title feat: L2ToL2CDM RelayedMessageGasReceipt + SentMessage ctx feat: L2ToL2CDM SentMessage ctx May 7, 2025
Copy link
Member

@clabby clabby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm for this change for the flexibility it gives us going forwards. Better to add it now than later, and the method of population is easy to change without breaking the event topic / schema in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants